-
Notifications
You must be signed in to change notification settings - Fork 121
[Woo POS][Surveys] Update final survey URLs and add track events #16248
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Woo POS][Surveys] Update final survey URLs and add track events #16248
Conversation
Generated by 🚫 Danger |
|
|
staskus
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ Notifications work
✅ Links works
✅ Tracks work
One comment:
CfT for Android mentioned that existing user notification is shown after the Woo app was opened and if POS was opened at least once. And iOS requires for the POS itself to be opened. Let's clarify that to align the platforms 👍
That's a good detail, will ask in the channel. Thanks! |

Closes WOOMOB-1497
Closes WOOMOB-1479
Description
In this PR we update the survey URLs to the real surveys, as well as we add tracks for when the notification is displayed, and when is tapped
POSNotificationScheduler.timeIntervalInSecondsto something like 5 seconds. And add the reset trigger to its initializer if you've tested surveys before:woo_pos_survey_potential_user_surveycase:woo_pos_survey_current_user_surveycase:TODO: